TextLayout: Clarify implementation of special case
authorDaniel Boles <dboles@src.gnome.org>
Wed, 1 Mar 2017 20:10:55 +0000 (20:10 +0000)
committerMatthias Clasen <mclasen@redhat.com>
Sun, 18 Feb 2018 03:18:13 +0000 (22:18 -0500)
commitdb4895fbe52c63fdcee60a35c64a7afbb353e243
tree8efd61c4fb6ef9e0b6ef0a7f20480899ddc4f876
parentf42ebef9d186ff87df2bbe0a37d7d14e58103577
TextLayout: Clarify implementation of special case

This exists to exit early for invisible lines. It attempts to use the
LineDisplay’s direction to create a corresponding PangoLayout. However,
the dir is not yet set by this point, & the display was new0()d, so its
dir is always 0 == TEXT_DIR_NONE. Thus, we always create an LTR layout.

Whatever the original intent, this outcome seems to be OK, so let’s make
the code say what it means, rather than using a misleading conditional.

https://bugzilla.gnome.org/show_bug.cgi?id=779099
gtk/gtktextlayout.c